Skip to content

Conversation

@NethmiRodrigo
Copy link
Contributor

This PR adds the configurations for the demo metadata from the ref app.

@NethmiRodrigo NethmiRodrigo requested a review from ibacher January 31, 2025 14:53
@NethmiRodrigo
Copy link
Contributor Author

@ibacher I am yet to test this out, and I'm not sure how to.

@NethmiRodrigo NethmiRodrigo changed the title (feat): Add demo data from the refapp (feat) O3-4429: Add demo data from the refapp Feb 4, 2025
@NethmiRodrigo
Copy link
Contributor Author

@dkayiwa @wikumChamith here is the PR to add the rest of the configurations to a demo content package

@dkayiwa
Copy link
Member

dkayiwa commented Feb 4, 2025

@NethmiRodrigo when setting up O3, what exact steps would i need to take advantage of this demo data package?

@NethmiRodrigo
Copy link
Contributor Author

@dkayiwa we'd have to publish a maven artifact by creating a bamboo plan and merge this in, and then we'd have to add this content package to the ref app's content.properties file (similar to how we did with the other ref app content package)

@jayasanka-sack
Copy link
Member

I created a bamboo plan here: https://ci.openmrs.org/browse/CH-CONREFD

Btw, @NethmiRodrigo did you test this out?

@NethmiRodrigo
Copy link
Contributor Author

Thank you @jayasanka-sack! Testing it out now, @wikumChamith was walking me through how to test it with the sdk

@NethmiRodrigo
Copy link
Contributor Author

@jayasanka-sack tested it out with the SDK and all configurations were loaded in correctly.

Copy link
Member

@jayasanka-sack jayasanka-sack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! Thanks for getting this done @NethmiRodrigo!

@jayasanka-sack jayasanka-sack merged commit ba5061d into main Feb 5, 2025
1 check passed
@jayasanka-sack jayasanka-sack deleted the feat/add-demo-data-from-refapp branch February 5, 2025 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants